Skip to content

Implement GDPatch modloader installers - #2303

Open
ethangreen-dev wants to merge 3 commits into
developfrom
gdpatch
Open

ethangreen-dev wants to merge 3 commits into
developfrom
gdpatch

Conversation

@ethangreen-dev

Copy link
Copy Markdown
Collaborator

Implements the modloader and mod installers for the GDPatch modloader.

https://github.com/GDPatch/GDPatch

@ethangreen-dev
ethangreen-dev requested a review from ebkr October 6, 2026 07:19
Comment thread src/installers/GDPatchInstaller.ts Outdated
Comment thread src/installers/GDPatchInstaller.ts Outdated

const fs = FsProvider.instance;

const windowsLoaderSrc = path.join(packagePath, "gdpatch_loader.dll");

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's probably wise of us to rename this to winmm.dll in the package itself to make extracting it manually user-proof; should this filename be updated in that case?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NotNite Probably? It would make sense for it to be easy to manually install the package.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We haven't packaged GDPatch just yet (we wanted preliminary mod manager support first), so let's assume the DLL will be named winmm.dll in the package and fix the path here.

Renaming files to .old leaves a folder without gdpatch_mod.toml, which
GDPatch logs as a broken mod. The marker was agreed with the GDPatch
maintainers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants